Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix some links #4733

Merged
merged 1 commit into from
Jul 28, 2023
Merged

Fix some links #4733

merged 1 commit into from
Jul 28, 2023

Conversation

philrz
Copy link
Contributor

@philrz philrz commented Jul 28, 2023

No description provided.

@philrz philrz requested a review from a team July 28, 2023 18:57
@philrz philrz self-assigned this Jul 28, 2023
@@ -1,7 +1,7 @@
## v1.9.0
* The [Zed Language Overview docs](docs/language/overview.md) have been split into multiple sections (#4576)
* Add support for [user-defined operators](docs/language/statements.md#operator-statements) (#4417, #4635, #4646, #4644, #4663, #4674, #4698, #4702, #4716)
* Add experimental support to the [`get` operator](https://github.com/brimdata/zed/blob/main/docs/language/operators/get.md) for customized methods, headers, and body (#4572)
* Add experimental support to the [`get` operator](docs/language/operators/get.md) for customized methods, headers, and body (#4572)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one was a straight up mistake by me. Normally I'm careful to always use relative links in the CHANGELOG, then when I post these as release notes in GitHub I morph them there to absolute links pointing to the docs site. I just missed this one.

@@ -25,7 +25,7 @@ in `jq`. If there's glaring functional omissions that are limiting your use of
`zq`, we welcome [contributions](../README.md#contributing).

* For the permutations with `ndjson` input the recommended approach for
[shaping Zeek NDJSON](https://zed.brimdata.io/docs/next/integrations/zeek/shaping-zeek-ndjson)
[shaping Zeek NDJSON](https://zed.brimdata.io/docs/integrations/zeek/shaping-zeek-ndjson)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change was expected. When I made the changes in #4694 I knew the Zeek docs were going to be gone from their prior home on main in the repo, but they weren't yet revealed under any tagged version in the proper docs site, so for the moment I had to point to next. Now we have the tagged v1.9.0 docs, so that means I can drop the next.

@philrz philrz merged commit 16214e2 into main Jul 28, 2023
4 checks passed
@philrz philrz deleted the link-fixes branch July 28, 2023 19:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants